Auto merge of #4329 - behnam:dev, r=matklad

[doc/manifest] Add build-dependencies to Dependency sections list

The `[build-dependencies]` section was missing from the Dependency
sections list, making it hard to find its existence and link to
`specifying-dependencies` page. This puts the section title in the list,
so a normal find works.

Background: https://github.com/rust-lang/cargo/issues/4309#issuecomment-317965620

[doc/manifest] Add build-dependencies to Dependency sections list

The `[build-dependencies]` section was missing from the Dependency
sections list, making it hard to find its existence and link to
`specifying-dependencies` page. This puts the section title in the list,
so a normal find works.

Background: https://github.com/rust-lang/cargo/issues/4309#issuecomment-317965620

Auto merge of #4317 - iKevinY:options-unit-struct, r=matklad

Convert Options struct into unit struct

Unit struct support was added in Docopt 0.8.1 via docopt/docopt.rs#217. Fixes #4174.

Convert Options struct into unit struct

Unit struct support was added in Docopt 0.8.1 via
docopt/docopt.rs#217. Fixes #4174.

Auto merge of #4305 - nodakai:fix-4278-tests-on-empty-ld_library_path, r=matklad

tests/build.rs: remove empty components from LD_LIBRARY_PATH to fix #4278

The tests intended to ensure Cargo wouldn't mistakenly add empty components to `LD_LIBRARY_PATH` (or its equivalents) but they could fail when the test runner already had empty components in `LD_LIBRARY_PATH` from their environment.

tests/build.rs: remove empty components from LD_LIBRARY_PATH

This is to amend PR #4278

The tests intended to ensure Cargo wouldn't mistakenly add empty
components to LD_LIBRARY_PATH (or its equivalents) but they could fail
when the test runner already had empty components in LD_LIBRARY_PATH
from their environment.

Signed-off-by: NODA, Kai <nodakai@gmail.com>

Auto merge of #4303 - bluetech:doc-env-cfg, r=alexcrichton

Document the CARGO_CFG_* environment variables - fixes #4302

In order to keep the paragraph brief, I have omitted:

- examples of actual variable names - though adding those might help google.
- mention of `rustc --print cfg` - I think the link to the reference is enough.
- reference to the [RFC](https://github.com/rust-lang/rfcs/blob/master/text/1721-crt-static.md#lowering-cfg-values-to-cargo-build-script-environment-variables) - I don't think it's needed.

Auto merge of #4308 - alexcrichton:bump, r=alexcrichton

Bump to 0.22.0 and update deps

Work around AppVeyor for now

Bump to 0.22.0 and update deps

Document the CARGO_CFG_* environment variables - fixes #4302

Auto merge of #4299 - brson:home, r=alexcrichton

Replace homedir definition with home crate

This pulls out the definitions of these functions into a single crate, [home](https://github.com/brson/home), to act as their canonical definition.

This has a bigger impact on rustup, since it is using  definitions of all these things, where cargo is just defining cargo_home in terms of std's wrong home_dir.

The [definition of cargo_home](https://github.com/brson/home/blob/master/src/lib.rs#L179) used here is slightly different than what cargo has today in that it contains a bit of defensive code to avoid allowing `CARGO_HOME` to be set to `.multirust/cargo` - this is backwards compatibility code from rustup.

Auto merge of #4298 - behnam:ignore, r=alexcrichton

[src/doc/manifest] Add section on Migrating to `gitignore`-like pattern matching

Tracking issue: https://github.com/rust-lang/cargo/issues/4268

[src/doc/manifest] Add section on Migrating to `gitignore`-like pattern matching

Tracking issue: https://github.com/rust-lang/cargo/issues/4268

Replace homedir definition with home crate

Auto merge of #4123 - alexcrichton:augment, r=matklad

Implement the [patch] section of the manifest

This is an implementation of [RFC 1969] which adds a new section to top-level
manifests: `[patch]`. This section allows you to augment existing sources with
new versions of crates, possibly replacing the versions that already exist in
the source. More details about this feature can be found in the RFC itself.

[RFC 1969]: https://github.com/rust-lang/rfcs/pull/1969

Implement the [patch] section of the manifest

This is an implementation of [RFC 1969] which adds a new section to top-level
manifests: `[patch]`. This section allows you to patch existing sources with
new versions of crates, possibly replacing the versions that already exist in
the source. More details about this feature can be found in the RFC itself.

[RFC 1969]: https://github.com/rust-lang/rfcs/pull/1969

Auto merge of #4270 - behnam:ignore, r=alexcrichton

[sources/path] Add gitignore-like pattern matching and warn on mismatches

Add gitignore-like pattern matching logic to `list_files()` and throw
warnings for paths getting different inclusion/exclusion results from
the old and the new methods.

Migration Tracking: <#4268>

Auto merge of #4295 - xftroxgpx:use_full_path_in_warning_for_issue4293, r=matklad

Fixes #4293

locally tested with:
`cargo test legacy_ many_crate_types_old_style_lib_location`

remove leftover { }

Fixes #4293

locally tested with:
cargo test legacy_ many_crate_types_old_style_lib_location

Auto merge of #4292 - xftroxgpx:unconfuse_bin_name, r=matklad

quote the binary name in the warning

to avoid confusion when the warning happens inside a workspace, example:
pwd is rustlearnage/rust_books/1_first_edition/closures/
and `cargo run` shows:
```
warning: path `src/main.rs` was erroneously implicitly accepted for binary match,
please set bin.path in Cargo.toml
```
(note that 'match' is not quoted, so it's confusing)

the 'match' project that it's referring to is in rustlearnage/rust_books/1_first_edition/match/
(note: Cargo.toml [workspace] is in rustlearnage/ )

quoting library and benchmark names in warning

quote the binary name in the warning

to avoid confusion when the warning happens inside a workspace, example:
pwd is rustlearnage/rust_books/1_first_edition/closures/
and `cargo run` shows:

warning: path `src/main.rs` was erroneously implicitly accepted for binary match,
please set bin.path in Cargo.toml

(note that 'match' is not quoted, so it's confusing)

the 'match' project that it's referring to is in rustlearnage/rust_books/1_first_edition/match/
(note: Cargo.toml [workspace] is in rustlearnage/ )

[sources/path] Add gitignore-like pattern matching and warn on mismatches

Add gitignore-like pattern matching logic to `list_files()` and throw
warnings for paths getting different inclusion/exclusion results from
the old and the new methods.

Migration Tracking: <https://github.com/rust-lang/cargo/issues/4268>

Auto merge of #4289 - johnthagen:usage-fix, r=alexcrichton

Add "uninstall" command to cargo --help

Closes #4285

Auto merge of #4288 - matklad:bench-bench, r=alexcrichton

Allow `src/bench.rs` as a benchmark for legacy reasons

closes #4287
cc #3947

Add "uninstall" command to cargo --help

Auto merge of #4286 - cbcoutinho:master, r=matklad

Fix typo in zsh _cargo completions

None

Allow `src/bench.rs` as a benchmark for legacy reasons

closes #4287
cc #3947

Fix typo in zsh _cargo completions

Auto merge of #4279 - tee-too:fix-3499, r=alexcrichton

Fix support of `[target.'cfg(...)']` syntax for rustc and rustdoc flags

Support of `[target.'cfg(...)']` for rustc and rustdoc flags is buggy.
This adds meaningful tests and fixes the issue.

Fixes #3499

Fix support of `[target.'cfg(...)']` syntax for rustc and rustdoc flags

Support of `[target.'cfg(...)']` for rustc and rustdoc flags is buggy.
This adds meaningful tests and fixes the issue.

Fixes #3499

Auto merge of #4278 - mbrubeck:path, r=alexcrichton

Don't push empty paths in LD_LIBRARY_PATH

Fixes #4277.  Note: I haven't written a test for this fix yet.

Don't push empty paths in LD_LIBRARY_PATH

Fixes #4277.

Auto merge of #4275 - alexcrichton:revert-makeflags, r=alexcrichton

Don't set MAKEFLAGS for build scripts

Closes #4156
Closes rust-lang/rust#42635

Don't set MAKEFLAGS for build scripts

Closes #4156
Closes rust-lang/rust#42635

Auto merge of #4274 - sfackler:patch-1, r=alexcrichton

Add CircleCI badge to manifest reference

rust-lang/crates.io#807

Add CircleCI badge to manifest reference

rust-lang/crates.io#807

Auto merge of #4272 - behnam:doc, r=alexcrichton

[doc/crates-io] Add text about packaging tests/benches

Fixes <https://github.com/rust-lang/cargo/issues/4262>

[doc/crates-io] Add text about packaging tests/benches

Fixes <https://github.com/rust-lang/cargo/issues/4262>

Auto merge of #4259 - matklad:conventions, r=alexcrichton

Conventions

r? @alexcrichton

I'd love to refactor our handing of inferring targets by convention, because it is super difficult to understand, and quite probably contains a couple of unintended/undocumented conventions (like `src/foo.rs` being a binary `foo` if there's no library in the package).

As a first step, I've just moved all the logic (600 loc) to a separate file from the `toml.rs` file, which used to be just **huge** (we even use in in IntelliJ Rust for performance testing :) ), and now it is "only" just above 1kloc :)

Auto merge of #4267 - natboehm:add-index-flag-search, r=matklad

Replace `cargo search --host` with `cargo search --index`

This change makes the command consistent with other versions of the same flag. `cargo search --host` is still supported, as well as `cargo search --index <index> --host <host>` in case of confusion. The three commands currently supported should produce the same behavior. All instances of `--host` are currently marked as deprecated, or for reuse as mentioned in issue #4208.

Fixes #4219

Modified tests to use '--index' instead of '--host', added test to support using '--host' with warning message, added test to support using both '--host' and '--index' with warning message for host

rename '--host' to '--index', add comments marked as TODO where host should be removed/repurposed

Auto merge of #4266 - nrc:force, r=alexcrichton

Add a way to force a unit of work to always be rebuilt

Add a way to force a unit of work to always be rebuilt

Rearrange code

Simplify

Use Path::join instead of `/`

While `/` do work on windows, the resulting path will have literal `/`
in its display.

Fix more tests

Use the same logic for benches

Warn about obsolete conventions for binary names

Remove Layout structure

Fix tests

Merge duplicate tests

Warn about legacy library paths

Specify conventions for benches, examples and tests only once

Infer benches, tests and examples without layout

Reduce code duplication in validation

Unify error message formatting

Unify parameter order

Reduce code duplication & fix error messages

Jettison normalize function and its closures!

Extract clean_benches function

Extract clean_tests function

Extract clean_examples function

Extract clean_bins function

Extract clean_library function

Don't store root in the layout

Unify naming for `package_root`

Superfluous pub

Restructure code

Make layout private to targets

Move code for inferring build targets to a separate file

Fix typo

Refactor: virtual manifest does not need layout

Simplify

Simplify

Make `Layout` a private implementation detail of toml

Auto merge of #4257 - alexcrichton:update, r=alexcrichton

Update dependencies with semver-compat version

Just a standard update

Update dependencies with semver-compat version

Just a standard update

Auto merge of #4248 - dethoter:issue-4228, r=alexcrichton

bench command supports --no-fail-fast flag.

Fix for #4228.

Auto merge of #4249 - tmccombs:zsh-test-update, r=alexcrichton

Add more options for cargo test zsh completion

Add more options for cargo test zsh completion

bench command supports --no-fail-fast flag.

Auto merge of #4246 - est31:master, r=Mark-Simulacrum

Let two further tests tolerate col info in panics

Needed by https://github.com/rust-lang/rust/pull/42938

I've now ripgrepped for "panicked at" and found no further
test that hardcodes the "filename:line$" format.

Let two further tests tolerate col info in panics

Needed by https://github.com/rust-lang/rust/pull/42938

I've now ripgrepped for "panicked at" and found no further
test that hardcodes the "filename:line$" format.

Auto merge of #4244 - est31:master, r=matklad

Let the cargo_bench_failing_test tolerate col info

Needed by https://github.com/rust-lang/rust/pull/42938

Let the cargo_bench_failing_test tolerate col info

Needed by https://github.com/rust-lang/rust/pull/42938

Auto merge of #4239 - Keruspe:master, r=alexcrichton

hash '__CARGO_DEFAULT_LIB_METADATA' in metadata for rustc

We already have __CARGO_DEFAULT_LIB_METADATA to force adding the hash of some metadata in the libraries file name when building rust.
For now, we only check if it is set or not.
This patch makes also use of its value to compute the hash. This way we'll be able to pass in the channel we're building to avoid hash collisions between channels.

Auto merge of #4214 - msehnout:master, r=matklad

Infer multi-file binaries like `src/bin/server/main.rs` by convention

This feature is described in issue #4086

Auto merge of #4238 - ashleygwilliams:policies, r=steveklabnik

updates to policies

based on convo with @withoutboats and @carols10cents

hash '__CARGO_DEFAULT_LIB_METADATA' in metadata for rustc

This will let us seed the release channel as part of the
metadata used for the hash to avoid conflicting hashes across
channels.

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

Add documentation for multi-file binary convention

Prevent unnecessary heap allocations and simplify condition

Auto merge of #4234 - michaelwoerister:smarter-cargo-incremental-2, r=alexcrichton

Set -Ccodegen-units=N for non-path dependencies when CARGO_INCREMENTAL is set.

This PR tweaks compilation for non-path dependencies when CARGO_INCREMENTAL is set. Before, `cargo` would compile such dependencies "just" non-incrementally while, with this PR, it will also set the number of CGUs to the number of CPU cores. Otherwise compiling those dependencies might take significantly longer (because of more expensive optimizations and less multi-threading) than compiling them incrementally -- which is what we wanted to avoid in the first place.

feat(policy): add information on security disclosures

fix(doc): clarify the content the code of conduct covers

"main" can be valid name for binary again, improve test